Skip to content

Show the template used in guide/block helpers to reduce confusion. - #187

Merged
jaylinski merged 1 commit into
handlebars-lang:masterfrom
reggietheroman:docs/guide-show-block-helpers-template
Aug 8, 2026
Merged

Show the template used in guide/block helpers to reduce confusion.#187
jaylinski merged 1 commit into
handlebars-lang:masterfrom
reggietheroman:docs/guide-show-block-helpers-template

Conversation

@reggietheroman

@reggietheroman reggietheroman commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

The current guide/block helpers section is a bit confusing as referenced in #186 and the PR #185.

@jaylinski is correct to point out that the descriptions reference to people is correct as seen in the example file used. However, the document is confusing to read with only the block helper preparation script and the generated output.

Showing the template from the example file makes the section flow better and easier to follow.

…to invoke the helper and lead to the final output.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the “Block Helpers” section in the guide to reduce reader confusion by showing the Handlebars template used in the interactive example, not just the helper registration script and output.

Changes:

  • Adds the example template to the “Block Helpers” walkthrough to improve readability and flow.
  • Splits the explanatory paragraph so the template is shown between the helper setup and the description of arguments/options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/guide/index.md
@jaylinski
jaylinski merged commit c89643c into handlebars-lang:master Aug 8, 2026
1 check passed
@jaylinski

Copy link
Copy Markdown
Member

I guess it's a little bit let confusing now. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants